Skip to content

Conversation

kurnakovv
Copy link

@kurnakovv kurnakovv commented Sep 20, 2025

#673 issue
Second try :)

Thanks so much for your help @jsquire and @christothes in #675
I fixed my mistake and ran this Invoke-CodeGen script.

P.S. I guess I shouldn't have included the changes in the tspCodeModel.json file, right? :))
image

@kurnakovv
Copy link
Author

@jsquire @christothes @joseharriaga @trrwilson @ShivangiReja Hello everyone! This parameter is quite urgent for us, as it's crucial for our upcoming release. If anyone could fit in a quick review, it would be incredibly helpful.

Thank you so much in advance for your time and assistance! ❤️

@christothes
Copy link
Collaborator

P.S. I guess I shouldn't have included the changes in the tspCodeModel.json file, right? :))

The tspCodeModel.json file changes any time generated code changes, so you should be able to merge with main and then just regenerate. In addition, please run /scripts/export-Api.ps1 to update the public API listing.

@kurnakovv kurnakovv force-pushed the feature-673-second-try branch from 3861db2 to 61540f5 Compare September 26, 2025 05:47
@kurnakovv
Copy link
Author

kurnakovv commented Sep 26, 2025

@christothes I run this script and got this error

&: D:\C#\openai-dotnet\scripts\Export-Api.ps1:218 Line | 218 | & genapi @genapiArgs | ~~~~~~ | The term 'genapi' is not recognized as a name of a cmdlet, function, script file, or executable program. Check | the spelling of the name, or if a path was included, verify that the path is correct and try again. Cleaning up D:\C#\openai-dotnet\api\OpenAI.netstandard2.0.cs...

I install this library https://github.com/MigleSur/GenAPI

And now got this

parse-options: unknown option -- assembly parse-options: unknown option -- output-path parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference ERROR: blast is not installed or not included in the path.

Can you help me, please? :)
What Am I doing wrong?

P.S. I'm a little confused because blast is something from Linux, although the scripts are for powershell (Windows)

P.P.S. Okay, I realized that this repository is something completely different from what he asks for, but in any case, I didn’t find on the Internet how to replace it + I also didn’t see any script with this name in the repository

@kurnakovv kurnakovv force-pushed the feature-673-second-try branch from 6c8a3fd to c8e8f2d Compare September 26, 2025 13:30
@kurnakovv
Copy link
Author

Ok, I finally solve the problem :)
Here right tool to run script https://github.com/dotnet/sdk/blob/main/src/Compatibility/GenAPI/README.md

@jsquire
Copy link
Collaborator

jsquire commented Sep 26, 2025

Thanks very much for your contribution, @kurnakovv. Would you please be so kind as to add a change log entry to capture this?

@kurnakovv
Copy link
Author

kurnakovv commented Sep 27, 2025

@christothes Thank you so much for your approval 👍

@jsquire Yes, no problem! I’ve updated the CHANGELOG file.
One small question: should I replace [Unreleased] with 2.6.0 (2025-09-27), or is versioning handled on your side?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants